From 6e0b445ee738cd829ed28c7532cded8afe25886d Mon Sep 17 00:00:00 2001 From: Juergen Gross Date: Thu, 20 Feb 2020 11:38:31 +0100 Subject: [PATCH] sched: don't disable interrupts all the time when dumping run-queues Having interrupts disabled all the time when running dump_runq() is not necessary. All the called functions are doing proper locking and disable interrupts if needed. Signed-off-by: Juergen Gross Reviewed-by: Dario Faggioli --- xen/common/sched/cpupool.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/xen/common/sched/cpupool.c b/xen/common/sched/cpupool.c index 476916c6ea..b7721b5d02 100644 --- a/xen/common/sched/cpupool.c +++ b/xen/common/sched/cpupool.c @@ -889,12 +889,10 @@ const cpumask_t *cpupool_valid_cpus(const struct cpupool *pool) void dump_runq(unsigned char key) { - unsigned long flags; s_time_t now = NOW(); struct cpupool **c; spin_lock(&cpupool_lock); - local_irq_save(flags); printk("sched_smt_power_savings: %s\n", sched_smt_power_savings? "enabled":"disabled"); @@ -914,7 +912,6 @@ void dump_runq(unsigned char key) schedule_dump(*c); } - local_irq_restore(flags); spin_unlock(&cpupool_lock); } -- 2.30.2